Skip to content

Conversation

@kurapativyshnavi
Copy link
Contributor

Add Cookie Statement page explaining Able Player's use of functional cookies

Add Cookie Statement page explaining Able Player's use of functional cookies
@rianrietveld
Copy link
Member

Thank you @kurapativyshnavi
I'll look into it on Monday

@rianrietveld rianrietveld moved this to PRs to review in WP A11y docs Sep 15, 2025
@rianrietveld rianrietveld linked an issue Sep 15, 2025 that may be closed by this pull request
Copy link
Member

@rianrietveld rianrietveld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for writing this.
I added some comments with the code
On the page https://wpaccessibility.org/docs/contribute/documentation/ you can read how the code of a page is set up.

@joedolson can you please review the content itself?
As you know better how the Able player is set up.

Update cookies.md based on review feedback
Copy link
Member

@joedolson joedolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs a few changes so that the topic of the page is about how the website uses cookies. A cookie statement needs to be about how the site uses cookies, and address the specific cookies and types of cookies used. Even though there is only one part of the site that uses cookies, we need to do everything in reference to the site holistically.

@yoren
Copy link
Collaborator

yoren commented Sep 18, 2025

Hey @kurapativyshnavi - You may want to merge changes from the latest main branch into your branches and see if that can properly trigger the PR Preview workflows, which can build up a preview version of the website.

Also cc @rianrietveld so once confirmed, we can close #100

@rianrietveld rianrietveld moved this from PRs to review to In Progress in WP A11y docs Sep 30, 2025
@rianrietveld
Copy link
Member

Hey @kurapativyshnavi
Do you still want to finish this by merging main into your PR?
If you do not have the time, please let me know, then I will merge main and finish the PR it for you.

@kurapativyshnavi
Copy link
Contributor Author

kurapativyshnavi commented Oct 15, 2025 via email

@rianrietveld rianrietveld self-requested a review October 15, 2025 09:38
@rianrietveld rianrietveld merged commit 82f66ec into wpaccessibility:main Oct 15, 2025
2 of 4 checks passed
@github-actions
Copy link
Contributor

PR Preview
Preview removed because the pull request was closed.
2025-10-15 09:40 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Topic] Cookie statement, because the able player uses them

5 participants